Skip to content

added architecture diagram and readme#11

Merged
djliden merged 4 commits intodatabricks-solutions:mainfrom
dmatrix:br_jsd_add_arch_image
Apr 6, 2026
Merged

added architecture diagram and readme#11
djliden merged 4 commits intodatabricks-solutions:mainfrom
dmatrix:br_jsd_add_arch_image

Conversation

@dmatrix
Copy link
Copy Markdown
Contributor

@dmatrix dmatrix commented Apr 3, 2026

  • added architecture diagram
  • minor tweak to readme

dmatrix added 3 commits April 2, 2026 17:21
Signed-off-by: Jules Damji <dmatrix@comcast.net>
Signed-off-by: Jules Damji <dmatrix@comcast.net>
… agents to the Supervisor

Signed-off-by: Jules Damji <dmatrix@comcast.net>
Copy link
Copy Markdown
Collaborator

@djliden djliden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The SVG looks good and the table format for the subagents is a nice upgrade.

One note on the new "Step 1: Create a SQL Warehouse" section: I think the intent is good (helping users find their warehouse ID), but the framing could cause some confusion:

  1. Most users already have a warehouse. Databricks auto-provisions a "Starter Warehouse" in every workspace, and many users will have others already. Framing this as "Create a SQL Warehouse" implies they always need to create a new one, which could lead to unnecessary warehouse proliferation.

  2. Genie requires Pro or Serverless specifically — Classic warehouses won't work. Worth mentioning if we're going to walk users through this.

The original README handled this by listing "a SQL Warehouse ID" in the prerequisites and showing it in the deploy command, which worked fine. If we want to be more helpful, I'd suggest either:

  • Keep it in Prerequisites with a brief note on where to find the warehouse ID (SQL Warehouses in the sidebar → click your warehouse → ID is in the URL or Connection Details tab), or
  • Reframe the step as "Step 1: Find your SQL Warehouse ID" with language like "Select an existing Pro or Serverless SQL warehouse (or create one if needed)" rather than leading with creation.

Minor other things I noticed:

  • "colonoies" → "colonies" in the subagent table

Signed-off-by: Jules Damji <dmatrix@comcast.net>
@djliden djliden merged commit c704e0a into databricks-solutions:main Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants